Skip to content

Comments

Comment PR with installer link on new build#265

Merged
gr5 merged 6 commits intomasterfrom
atsju-patch-1
Nov 9, 2025
Merged

Comment PR with installer link on new build#265
gr5 merged 6 commits intomasterfrom
atsju-patch-1

Conversation

@atsju
Copy link
Collaborator

@atsju atsju commented Nov 9, 2025

No description provided.

@github-actions
Copy link

github-actions bot commented Nov 9, 2025

🚀 New build available for commit ``
Download installer here

@github-actions
Copy link

github-actions bot commented Nov 9, 2025

🚀 New build available for commit c07445e
Download installer here

@github-actions
Copy link

github-actions bot commented Nov 9, 2025

🚀 New build available for commit 9057a57
Download installer here

@atsju atsju requested review from githubdoe and gr5 November 9, 2025 10:18
@atsju atsju changed the title [under development] Comment PR with installer link on new build Comment PR with installer link on new build Nov 9, 2025
@atsju
Copy link
Collaborator Author

atsju commented Nov 9, 2025

Ready for review. Please merge when you are OK with it.

We will now get a direct link to installer for each successful Qt6 build in a PR.

@gr5
Copy link
Collaborator

gr5 commented Nov 9, 2025

Haven't we always gotten that? I just go to "actions" on github, find the one for the PR I care about, go to the non-qt5 windows build and run the installer.

@gr5 gr5 merged commit 747b581 into master Nov 9, 2025
14 checks passed
@gr5
Copy link
Collaborator

gr5 commented Nov 9, 2025

I merged it.

@atsju atsju deleted the atsju-patch-1 branch November 9, 2025 14:16
@atsju
Copy link
Collaborator Author

atsju commented Nov 9, 2025

Haven't we always gotten that? I just go to "actions" on github, find the one for the PR I care about, go to the non-qt5 windows build and run the installer.

What's new is this automatic comment in PR itself: #265 (comment)

Dale did't know how to get the installer by himself and I must admit it's not that straightforward. Now you get the installer in a single click.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants